Skip to content

Document string ToString translation - #476

Open
roji wants to merge 1 commit into
mainfrom
roji-ef-provider-documentation
Open

roji wants to merge 1 commit into
mainfrom
roji-ef-provider-documentation

Conversation

@roji

@roji roji commented Oct 20, 2026

Copy link
Copy Markdown
Member

Document parameterless string.ToString() as a SQL identity expression, preserving nulls, following merged npgsql/efcore.pg#4047.

This is a bug-fix reference update, not a new 11.0 feature announcement. The availability note reflects the merged main/11.0 implementation; the 8.0, 9.0 and 10.0 backports are still pending and are not advertised as available.

The other nine coordinated source workstreams are still unmerged. Their documentation coverage is tracked in the session and will be updated on confirmed merge outcomes; proposed features are deliberately excluded here. Internal mapping, generated-column ordering, connection-ownership and diagnostic fixes do not currently require public documentation changes.

Validation: npx --yes markdownlint-cli conceptual/EFCore.PG/mapping/translations.md and git diff --check.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot-Session: b4f6a4da-cf90-40f2-a1e2-2a54035bd9df
Copilot AI balanced review requested due to automatic review settings October 20, 2026 12:14

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The focused documentation update matches the merged implementation, with no unresolved findings.

No open issues 🎉

What changed in this PR

Documents parameterless string.ToString() in the EF Core PostgreSQL translation reference.

Changes:

  • Records the SQL identity translation and null preservation.
  • Notes availability in 11.0 without advertising pending backports.
File Description
conceptual/​EFCore.PG/​mapping/​translations.md Adds the string.ToString() translation and availability note.

Review effort: Balanced

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants